Add support for SDL2 Renderer backend #193
Conversation
Hm... This prompts us to use a distinct flag for the renderer. Something like |
…ot have SDL_RenderGeometry
|
Nice catch! 84ae150 has the new flag. |
| @@ -94,6 +94,15 @@ flag sdl | |||
| manual: | |||
| True | |||
|
|
|||
| flag sdl-renderer | |||
| description: | |||
| Enable SDL Renderer backend (requires the SDL_RenderGeometry function available in SDL 2.0.18+). | |||
There was a problem hiding this comment.
The ImGui preprocessor error calls for 2.0.17+, but I opted for 2.0.18+ here as that's when the official docs and release notes indicate SDL_RenderGeometry was added.
My understanding is that SDL uses the odd version numbers for releases that might not be proven to be stable, so it may likely have been added in 2.0.17, but it seems users won't find much in the way of documentation for that version, so 2.0.18+ will hopefully avoid confusion.
Closes #191. Here is a gif showing the added example program using the SDL2 renderer:
There are a couple stray commits included that add a
withCloseableWindowvariant andframerateas the SDL2 Renderer example uses these bits. Let me know if there is a preference on splitting those out into separate PRs.